Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[13.0][FIX] storage media listed on product variant #116

Merged
merged 1 commit into from
Sep 9, 2021

Conversation

TDu
Copy link
Member

@TDu TDu commented Sep 3, 2021

This PR fixes the media displayed on the product variant form in the media
tab.

The change on the depends force the recomputation when an attibute value
is changed on the product.media.relation.

The other change fixes the attributes value that are tested to assign
a media on a variant.
It is the attribute set on the product variant that should be used and
not all attribute values assigned to the product template.

This PR fixes the media displayed on the product variant form in the media
tab.

The change on the depends force the recomputation when an attibute value
is changed on the product.media.relation.

The other change fixes the attributes value that are tested to assign
a media on a variant.
It is the attribute set on the product variant that should be used and
not all attribute values assigned to the product template.
@OCA-git-bot
Copy link
Contributor

Hi @TDu! Thank you very much for this contribution. As the addon you are improving does not have a declared maintainer, I take the opportunity to mention that you can consider adopting it. To do so, please read the maintainer role description, and, if interested, create a pull request to add your GitHub login to the maintainers key of the addon manifest.

@simahawk
Copy link
Contributor

simahawk commented Sep 9, 2021

/ocabot merge patch

@TDu would you mind porting this to v14?

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 13.0-ocabot-merge-pr-116-by-simahawk-bump-patch, awaiting test results.

@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 8a21b2f. Thanks a lot for contributing to OCA. ❤️

TDu added a commit to camptocamp/storage that referenced this pull request Sep 9, 2021
This PR fixes the media displayed on the product variant form in the media
tab.

The change on the depends force the recomputation when an attibute value
is changed on the product.media.relation.

The other change fixes the attributes value that are tested to assign
a media on a variant.
It is the attribute set on the product variant that should be used and
not all attribute values assigned to the product template.

This is a forward port from:

* OCA#116
TDu added a commit to camptocamp/storage that referenced this pull request Sep 9, 2021
This PR fixes the media displayed on the product variant form in the media
tab.

The change on the depends force the recomputation when an attibute value
is changed on the product.media.relation.

The other change fixes the attributes value that are tested to assign
a media on a variant.
It is the attribute set on the product variant that should be used and
not all attribute values assigned to the product template.

Forward port from:

* OCA#116
@TDu
Copy link
Member Author

TDu commented Sep 9, 2021

/ocabot merge patch

@TDu would you mind porting this to v14?

There it is #118

JasminSForgeFlow pushed a commit to ForgeFlow/storage that referenced this pull request Oct 26, 2023
This PR fixes the media displayed on the product variant form in the media
tab.

The change on the depends force the recomputation when an attibute value
is changed on the product.media.relation.

The other change fixes the attributes value that are tested to assign
a media on a variant.
It is the attribute set on the product variant that should be used and
not all attribute values assigned to the product template.

Forward port from:

* OCA#116
LucasTran380381 pushed a commit to LucasTran380381/storage that referenced this pull request Jan 7, 2025
This PR fixes the media displayed on the product variant form in the media
tab.

The change on the depends force the recomputation when an attibute value
is changed on the product.media.relation.

The other change fixes the attributes value that are tested to assign
a media on a variant.
It is the attribute set on the product variant that should be used and
not all attribute values assigned to the product template.

Forward port from:

* OCA#116
sebalix pushed a commit to camptocamp/storage that referenced this pull request Jan 22, 2025
This PR fixes the media displayed on the product variant form in the media
tab.

The change on the depends force the recomputation when an attibute value
is changed on the product.media.relation.

The other change fixes the attributes value that are tested to assign
a media on a variant.
It is the attribute set on the product variant that should be used and
not all attribute values assigned to the product template.

Forward port from:

* OCA#116
sebalix pushed a commit to camptocamp/storage that referenced this pull request Feb 20, 2025
This PR fixes the media displayed on the product variant form in the media
tab.

The change on the depends force the recomputation when an attibute value
is changed on the product.media.relation.

The other change fixes the attributes value that are tested to assign
a media on a variant.
It is the attribute set on the product variant that should be used and
not all attribute values assigned to the product template.

Forward port from:

* OCA#116
sebalix pushed a commit to camptocamp/storage that referenced this pull request Mar 4, 2025
This PR fixes the media displayed on the product variant form in the media
tab.

The change on the depends force the recomputation when an attibute value
is changed on the product.media.relation.

The other change fixes the attributes value that are tested to assign
a media on a variant.
It is the attribute set on the product variant that should be used and
not all attribute values assigned to the product template.

Forward port from:

* OCA#116
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants